Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge in new changes to AntPost branch #330

Merged
merged 19 commits into from
Dec 16, 2024
Merged

Merge in new changes to AntPost branch #330

merged 19 commits into from
Dec 16, 2024

Conversation

Bradley-Karat
Copy link
Collaborator

Many changes to master since the Anterior-Posterior branch was started. Before further development want to merge these changes in.

jordandekraker and others added 19 commits July 22, 2024 13:56
* bigbrain template laplace coords added

* bugfix for using L or R template injection

* lint

---------

Co-authored-by: Jordan DeKraker <[email protected]>
* Wet-run test data

* typos & clarifications
* fix dependency versions for snakebids, pulp

* poetry lock

* Revert "poetry lock"

This reverts commit b2f8809.

* redo poetry lock

* python >3.8

* dropped testing for python <3.8

---------

Co-authored-by: Jordan DeKraker <[email protected]>
* moved import to where it is needed

---------

Co-authored-by: Jordan DeKraker <[email protected]>
* Update Dockerfile to latest version with dependencies

* Update Dockerfile

using the new repo isntead of the old!
run_inference was only accepting a single input image, this makes it
accept a list of images too, and makes it so --force-nnunet-model T1T2w grabs both T1w and T2w as inputs.
this didn't affect any workflow computations, just the subsequent
dscalar.nii and spec file that would have the wrong curvature map
FIX: input images when using T1T2w model
FIX: hipp curvature was being used always, instead of wildcard
@akhanf akhanf merged commit c649755 into AntPost Dec 16, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants